Skip to content

docs: DOC-1310: improve listener documentation#7997

Merged
margaretkennedy merged 4 commits into
deephaven:mainfrom
margaretkennedy:doc-1310
May 20, 2026
Merged

docs: DOC-1310: improve listener documentation#7997
margaretkennedy merged 4 commits into
deephaven:mainfrom
margaretkennedy:doc-1310

Conversation

@margaretkennedy

Copy link
Copy Markdown
Contributor

No description provided.

…ns and error handling guidance

Updated Groovy and Python table listener guides with explicit type annotations, improved error handling examples, and clarified documentation. Changed error handling example to throw exceptions rather than catch them, matching best practices for listener failures. Added missing imports, fixed variable scoping, and updated method descriptions for clarity. Corrected timing examples and improved code
@github-actions

github-actions Bot commented May 7, 2026

Copy link
Copy Markdown
Contributor

Deploying docs previews for c1d9dbf (available for 14 days)

Python
Groovy

@margaretkennedy margaretkennedy requested a review from rbasralian May 7, 2026 19:41

@rbasralian rbasralian left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

couple tiny tweaks (use Class<?> instead of Class) but lgtm otherwise

Comment thread docs/groovy/how-to-guides/table-listeners-groovy.md Outdated
Comment thread docs/groovy/how-to-guides/table-listeners-groovy.md Outdated
Comment thread docs/groovy/how-to-guides/table-listeners-groovy.md Outdated
Comment thread docs/groovy/how-to-guides/table-listeners-groovy.md Outdated
Comment thread docs/groovy/how-to-guides/table-listeners-groovy.md Outdated
Comment thread docs/groovy/how-to-guides/table-listeners-groovy.md Outdated
margaretkennedy and others added 2 commits May 14, 2026 15:51
Co-authored-by: rbasralian <rbasralian@gmail.com>
@margaretkennedy margaretkennedy enabled auto-merge (squash) May 20, 2026 14:21
@margaretkennedy margaretkennedy merged commit f3a7e06 into deephaven:main May 20, 2026
24 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators May 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants